-
Notifications
You must be signed in to change notification settings - Fork 181
FRC-0104: add ChainGetFinalizedTipSet #1199
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Adds `ChainGetFinalizedTipSet` given it's addition to Lotuvs `/v1` in filecoin-project/lotus#13299
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. @AlexeyKrasnoperov could you help @BigLep with updating the spec once we agree this is good to go?
| ChainGetBlock | State query | Returns the block with the specified CID. | | ||
| ChainGetBlockMessages | State query | Returns all messages from the specified block. | | ||
| ChainGetEvents | State query | Returns the events under the given event AMT root CID. | | ||
| ChainGetFinalizedTipSet | State query | Returns the returns the latest F3 finalized tipset, or falls back to EC finality if F3 is not operational on the node or if the F3 finalized tipset is further back than EC finalized tipset. | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| ChainGetFinalizedTipSet | State query | Returns the returns the latest F3 finalized tipset, or falls back to EC finality if F3 is not operational on the node or if the F3 finalized tipset is further back than EC finalized tipset. | | |
| ChainGetFinalizedTipSet | State query | Returns the latest F3 finalized tipset, or falls back to EC finality if F3 is not operational on the node or if the F3 finalized tipset is further back than EC finalized tipset. | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm aside from Alexey's suggestion
Adds
ChainGetFinalizedTipSet
to FRC-0104 given it's addition to Lotuvs/v1
in filecoin-project/lotus#13299.If this idea is accepted, then can do the other work for this FRC to document its acceptance like updating https://github.com/filecoin-project/FIPs/tree/master/resources/frc-0104